Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to ocpp 2.0.0 to fix blocking #1423

Merged
merged 25 commits into from
Jan 5, 2025
Merged

Update to ocpp 2.0.0 to fix blocking #1423

merged 25 commits into from
Jan 5, 2025

Conversation

drc38
Copy link
Collaborator

@drc38 drc38 commented Dec 9, 2024

Fixes #1444, fixes #1402, fixes #1371, fixes #1461

@drc38 drc38 had a problem deploying to continuous-integration December 9, 2024 10:02 — with GitHub Actions Failure
@drc38
Copy link
Collaborator Author

drc38 commented Dec 9, 2024

@ars3niy can you take a look at why v2.0.1 tests fail with the latest ocpp release candidate please.

@drc38 drc38 temporarily deployed to continuous-integration December 12, 2024 07:12 — with GitHub Actions Inactive
Copy link

codecov bot commented Dec 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.69%. Comparing base (01b7cb1) to head (c0a17ae).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1423   +/-   ##
=======================================
  Coverage   94.69%   94.69%           
=======================================
  Files          12       12           
  Lines        1943     1943           
=======================================
  Hits         1840     1840           
  Misses        103      103           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@drc38 drc38 had a problem deploying to continuous-integration December 12, 2024 07:25 — with GitHub Actions Failure
This reverts commit 3c5c4bd.
@drc38 drc38 temporarily deployed to continuous-integration December 12, 2024 07:37 — with GitHub Actions Inactive
@drc38 drc38 temporarily deployed to continuous-integration December 12, 2024 07:47 — with GitHub Actions Inactive
@drc38 drc38 requested a review from lbbrhzn December 15, 2024 18:26
@ars3niy
Copy link
Contributor

ars3niy commented Dec 16, 2024

@ars3niy can you take a look at why v2.0.1 tests fail with the latest ocpp release candidate please.

It failed because _get_inventory from ocppv201.py appeared to return before NotifyReport without "tbc":true was received. How this was possible, I have no idea right now, it does not happen when I run locally.

@ars3niy
Copy link
Contributor

ars3niy commented Dec 16, 2024

This is where it waits for "tbc":true to disappear:

        if not kwargs.get("tbc", False):
            self._wait_inventory.set()

It is as if it did not see the tbc parameter when it was there.

@drc38
Copy link
Collaborator Author

drc38 commented Dec 16, 2024

This is where it waits for "tbc":true to disappear:

        if not kwargs.get("tbc", False):
            self._wait_inventory.set()

It is as if it did not see the tbc parameter when it was there.

It looks like it has something to do with threading and schema validation given disabling using the thread executor pool for validation resolves the error.

Could you please update your code to use the non-deprecated Action enums eg Action.get_base_report vs Action.GetBaseReport as they are going to be removed in release 2.0.0

@ars3niy
Copy link
Contributor

ars3niy commented Dec 16, 2024

Incidentally, since you consider this code to be mine, please send e-mail to arseniy.lartsev at qodes.se if there are any further issues. I may not immediately notice or else quickly forget things otherwise.

@drc38
Copy link
Collaborator Author

drc38 commented Dec 16, 2024

Incidentally, since you consider this code to be mine, please send e-mail to arseniy.lartsev at qodes.se if there are any further issues. I may not immediately notice or else quickly forget things otherwise.

Thanks @ars3niy, I get emailed directly when mentioned in a github comment do you not? Is there some other way to include your email in a comment to trigger an email?

@drc38 drc38 temporarily deployed to continuous-integration December 20, 2024 09:25 — with GitHub Actions Inactive
@lbbrhzn lbbrhzn temporarily deployed to continuous-integration December 25, 2024 12:19 — with GitHub Actions Inactive
@drc38 drc38 temporarily deployed to continuous-integration December 25, 2024 16:01 — with GitHub Actions Inactive
@lbbrhzn lbbrhzn temporarily deployed to continuous-integration December 25, 2024 16:47 — with GitHub Actions Inactive
@drc38 drc38 temporarily deployed to continuous-integration December 26, 2024 08:10 — with GitHub Actions Inactive
@drc38 drc38 changed the title Prerelease Update to ocpp 2.0.0rc3 to fix blocking Dec 26, 2024
@drc38 drc38 had a problem deploying to continuous-integration January 3, 2025 07:03 — with GitHub Actions Failure
@drc38 drc38 had a problem deploying to continuous-integration January 3, 2025 07:04 — with GitHub Actions Failure
@drc38 drc38 changed the title Update to ocpp 2.0.0rc3 to fix blocking Update to ocpp 2.0.0 to fix blocking Jan 3, 2025
@drc38 drc38 had a problem deploying to continuous-integration January 3, 2025 10:34 — with GitHub Actions Failure
@drc38 drc38 had a problem deploying to continuous-integration January 3, 2025 10:41 — with GitHub Actions Failure
@drc38 drc38 had a problem deploying to continuous-integration January 3, 2025 10:51 — with GitHub Actions Failure
@drc38 drc38 had a problem deploying to continuous-integration January 3, 2025 11:41 — with GitHub Actions Failure
@drc38 drc38 had a problem deploying to continuous-integration January 3, 2025 11:45 — with GitHub Actions Failure
@drc38 drc38 temporarily deployed to continuous-integration January 3, 2025 11:53 — with GitHub Actions Inactive
@drc38 drc38 temporarily deployed to continuous-integration January 4, 2025 10:20 — with GitHub Actions Inactive
@lbbrhzn lbbrhzn temporarily deployed to continuous-integration January 5, 2025 09:31 — with GitHub Actions Inactive
@lbbrhzn lbbrhzn temporarily deployed to continuous-integration January 5, 2025 09:34 — with GitHub Actions Inactive
@lbbrhzn lbbrhzn merged commit cd28cef into main Jan 5, 2025
7 checks passed
@lbbrhzn lbbrhzn deleted the prerelease branch January 5, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants